From e919fb2ac32a27c262fee8a40c044008b3f8fd09 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Roger=20Pau=20Monn=C3=A9?= Date: Wed, 22 Jul 2015 16:47:09 +0200 Subject: [PATCH] replace non-POSIX error codes MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Some DOMCTLs returned non-POSIX error codes, replace them with POSIX compilant values instead. EBADRQC and EBADSLT are replaced by EDOM, while EUSERS is replaced with EOVERFLOW. Signed-off-by: Roger Pau Monné Replace two EDOM uses. Reviewed-by: Jan Beulich Acked-by: George Dunlap Release-acked-by: Wei Liu --- xen/arch/x86/mm/paging.c | 2 +- xen/common/domain.c | 4 ++-- xen/common/hvm/save.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/xen/arch/x86/mm/paging.c b/xen/arch/x86/mm/paging.c index 708915515e..942d8491fc 100644 --- a/xen/arch/x86/mm/paging.c +++ b/xen/arch/x86/mm/paging.c @@ -766,7 +766,7 @@ long paging_domctl_continuation(XEN_GUEST_HANDLE_PARAM(xen_domctl_t) u_domctl) if ( op.interface_version != XEN_DOMCTL_INTERFACE_VERSION || op.cmd != XEN_DOMCTL_shadow_op ) - return -EBADRQC; + return -EOPNOTSUPP; d = rcu_lock_domain_by_id(op.domain); if ( d == NULL ) diff --git a/xen/common/domain.c b/xen/common/domain.c index 8efef5c179..791166b06c 100644 --- a/xen/common/domain.c +++ b/xen/common/domain.c @@ -900,7 +900,7 @@ int vcpu_pause_by_systemcontroller(struct vcpu *v) new = old + 1; if ( new > 255 ) - return -EUSERS; + return -EOVERFLOW; prev = cmpxchg(&v->controller_pause_count, old, new); } while ( prev != old ); @@ -980,7 +980,7 @@ int __domain_pause_by_systemcontroller(struct domain *d, * toolstack overflowing d->pause_count with many repeated hypercalls. */ if ( new > 255 ) - return -EUSERS; + return -EOVERFLOW; prev = cmpxchg(&d->controller_pause_count, old, new); } while ( prev != old ); diff --git a/xen/common/hvm/save.c b/xen/common/hvm/save.c index da6e668982..c9882e5783 100644 --- a/xen/common/hvm/save.c +++ b/xen/common/hvm/save.c @@ -114,7 +114,7 @@ int hvm_save_one(struct domain *d, uint16_t typecode, uint16_t instance, uint32_t off; const struct hvm_save_descriptor *desc; - rv = -EBADSLT; + rv = -ENOENT; for ( off = 0; off < (ctxt.cur - sizeof(*desc)); off += desc->length ) { desc = (void *)(ctxt.data + off); -- 2.30.2